Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

152 implementierung wahlvorstandsdaten #315

Merged
merged 43 commits into from
Jun 21, 2024
Merged

152 implementierung wahlvorstandsdaten #315

merged 43 commits into from
Jun 21, 2024

Conversation

MrSebastian
Copy link
Member

@MrSebastian MrSebastian commented Jun 19, 2024 •
edited
Loading

Beschreibung:

  • Implementierung Persistierung von Wahlvorstand
  • Die API wurde etwas verändert da ich der Meinung bin dass dies zu einer Verbesserung führt
    • PUT statt POST beim Speichern
    • neue Subressource anwesenheit bei Wahlvorstand da nur ein Teil der Daten aktualisiert wird
    • anwesenheitbeginn ist beim Lesen der Wahlvorstandes nicht von Bedeutung. Die Information beschreibt beim Speichern der Anwesenheit zu welchem Zeitpunkt der Zustand mit den Anwesenheiten gilt
    • die konkreten Funktionen die ein Wahlvorstandsmitglied haben kann wurde in die REST-API mit aufgenommen durch ein ENUM

Voraussetzung

Definition of Done (DoD):

Backend

  • Doku aktualisiert
  • Swagger-API vollständig
  • Unit-Tests gepflegt
  • Integrationstests gepflegt
  • Beispiel-Requests gepflegt

Referenzen1:

Verwandt mit Issue #

Closes #152

Footnotes

  1. Nicht zutreffende Referenzen vor dem Speichern entfernen ↩

@MrSebastian
draft impl for get and post
67ce338
@MrSebastian
add example requests for wahlvorstaende
5dac61e
@MrSebastian
update keycloak config for eai wahlvorstaende
66843d5
@MrSebastian
update example requests with invalid requests
bdcf03a
@MrSebastian
add global exception handler
54632f4
@MrSebastian
use ExceptionDataWrapper
4aee078
@MrSebastian
remove auto validation
6ae6388
@MrSebastian
add validator and udpate authority to latest used
529a324
@MrSebastian
preauthorized removed from controller
3e3ab51
@MrSebastian
update SecurityConfigurationTest with wahlvorstand
f8409f4
@MrSebastian
create wahlvorstand controller unit test
c756413
@MrSebastian
define authorities for wahlvorstaende service
c8e30e2
@MrSebastian
wahlvorstand controller integration test for load
1578309
@MrSebastian
move mapper to service package
e666d6c
@MrSebastian
create mapper test
ae1b89a
@MrSebastian
use optional in service
35f0013
@MrSebastian
use lambda expression instead of statement
1c9ed71
@MrSebastian
created tests for wahlvorstandservice
8282585
@MrSebastian
remove unintended noSec and randomPort of test
2299985
@MrSebastian
all constants with all authorities required to access service
d30355f
@MrSebastian
create WahlvorstandServiceSecurityTest
d666cf2
@MrSebastian
add lombok builder to dtos
a948e67
@MrSebastian
add validator tests
350600c
@MrSebastian
implement 404 on loadWahlvorstand
d3b0934
@MrSebastian
404 when wahlvorstand to update doesnt exists
6dab999
@MrSebastian
add tests for setanwesenheit
0fdc552
@MrSebastian
spotless:apply
2aacae9
@MrSebastian
change resource for anwesenheit
236fd6f
@MrSebastian
use put for update instead of post
3990e45
@MrSebastian
create test for global exception handler
bec3f6e
@MrSebastian MrSebastian self-assigned this Jun 19, 2024
@MrSebastian MrSebastian linked an issue Jun 19, 2024 that may be closed by this pull request
Implementierung Wahlvorstandsdaten #152
Closed
2 tasks
@MrSebastian MrSebastian marked this pull request as ready for review June 19, 2024 07:39
@dragonfly28 dragonfly28 self-requested a review June 20, 2024 07:19
dragonfly28
dragonfly28 approved these changes Jun 21, 2024
View reviewed changes
...c/main/java/de/muenchen/oss/wahllokalsystem/eaiservice/domain/wahlvorstand/Wahlvorstand.java Outdated Show resolved Hide resolved
...ava/de/muenchen/oss/wahllokalsystem/eaiservice/domain/wahlvorstand/WahlvorstandFunktion.java Outdated Show resolved Hide resolved
...va/de/muenchen/oss/wahllokalsystem/eaiservice/domain/wahlvorstand/Wahlvorstandsmitglied.java Outdated Show resolved Hide resolved
...ava/de/muenchen/oss/wahllokalsystem/eaiservice/rest/common/exception/ExceptionConstants.java Outdated Show resolved Hide resolved
...hen/oss/wahllokalsystem/eaiservice/rest/wahlvorstand/dto/WahlvorstandsaktualisierungDTO.java Outdated Show resolved Hide resolved
...e/muenchen/oss/wahllokalsystem/eaiservice/rest/wahlvorstand/dto/WahlvorstandFunktionDTO.java Outdated Show resolved Hide resolved
...a/de/muenchen/oss/wahllokalsystem/eaiservice/service/wahlvorstand/WahlvorstandValidator.java Outdated Show resolved Hide resolved
@dragonfly28
Copy link
Contributor

Beim Starten mit "./runLocalNoSecurity.sh" erhalte ich folgenden Fehler (Docker-Keycloak/Oracle-DB läuft):
grafik

Unter db/migrations/oracle finde ich auch kein File für den Wahlvorstand...

@MrSebastian
Copy link
Member Author

Beim Starten mit "./runLocalNoSecurity.sh" erhalte ich folgenden Fehler (Docker-Keycloak/Oracle-DB läuft): grafik

Unter db/migrations/oracle finde ich auch kein File für den Wahlvorstand...

Oracle files sind ergänzt: af55421

@MrSebastian MrSebastian merged commit 6dfb8da into dev Jun 21, 2024
3 checks passed
@MrSebastian MrSebastian deleted the 152-implementierung-wahlvorstandsdaten branch June 21, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dragonfly28 dragonfly28 dragonfly28 approved these changes

Assignees

@MrSebastian MrSebastian

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implementierung Wahlvorstandsdaten
2 participants
@MrSebastian @dragonfly28